Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hareflow] Add hareflow #24903

Merged
merged 6 commits into from
Jun 6, 2022
Merged

[hareflow] Add hareflow #24903

merged 6 commits into from
Jun 6, 2022

Conversation

yrivardmulrooney
Copy link
Contributor

@yrivardmulrooney yrivardmulrooney commented May 24, 2022

Describe the pull request
Add hareflow to ports

@ghost
Copy link

ghost commented May 24, 2022

CLA assistant check
All CLA requirements met.

@yrivardmulrooney yrivardmulrooney marked this pull request as draft May 24, 2022 19:16
@yrivardmulrooney yrivardmulrooney marked this pull request as ready for review May 24, 2022 20:42
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where you should check the license field.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/hareflow/vcpkg.json

Valid values for the license field can be found in the documentation

scripts/ci.baseline.txt Outdated Show resolved Hide resolved
ports/hareflow/vcpkg.json Show resolved Hide resolved
ports/hareflow/portfile.cmake Outdated Show resolved Hide resolved
ports/hareflow/portfile.cmake Outdated Show resolved Hide resolved
@Adela0814 Adela0814 added category:new-port The issue is requesting a new library to be added; consider making a PR! requires:author-response labels May 25, 2022
@yrivardmulrooney
Copy link
Contributor Author

Hey, thank you for the review 👍 , I added what was requested and dropped the exclusion of x86-windows. I don't think I need a "supports" section in vcpkg.json now right?

ports/hareflow/portfile.cmake Outdated Show resolved Hide resolved
@Adela0814
Copy link
Contributor

Hey, thank you for the review 👍 , I added what was requested and dropped the exclusion of x86-windows. I don't think I need a "supports" section in vcpkg.json now right?

Yes!

Copy link
Contributor

@ras0219-msft ras0219-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting this PR!

ports/hareflow/vcpkg.json Outdated Show resolved Hide resolved
@Adela0814 Adela0814 linked an issue Jun 1, 2022 that may be closed by this pull request
@yrivardmulrooney
Copy link
Contributor Author

Dropped the feature and made openssl/boost hard dependencies

@Adela0814 Adela0814 added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Jun 6, 2022
@ras0219-msft ras0219-msft merged commit 7ff57df into microsoft:master Jun 6, 2022
@ras0219-msft
Copy link
Contributor

Thanks for the PR!

@klalumiere klalumiere deleted the hareflow branch June 14, 2022 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[New Port Request] hareflow
4 participants